Skip to content

Support not cloning into repo subfolder, using env var for cred helper#15

Merged
Chuxel merged 2 commits into
mainfrom
chuxel/fix-git
Jun 10, 2026
Merged

Support not cloning into repo subfolder, using env var for cred helper#15
Chuxel merged 2 commits into
mainfrom
chuxel/fix-git

Conversation

@Chuxel

@Chuxel Chuxel commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

Adds two features some clients need to be able to take advantage of: not cloning into repo subfolder and using env var for cred helper

Copilot AI review requested due to automatic review settings June 10, 2026 19:11
@Chuxel Chuxel requested a review from a team as a code owner June 10, 2026 19:11

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR extends the SDK’s Git utilities to support two client needs: cloning directly into a provided directory (without auto-appending owner/repo) and optionally configuring Git credential helper to reference a token via an environment variable instead of writing the token into .git/config.

Changes:

  • Added excludeRepoSubfolder to allow cloning into cloneDir directly.
  • Added credentialHelperEnvVar to optionally reference credentials via an env var in credential.helper.
  • Bumped package version from 0.1.0 to 0.2.0.
Show a summary per file
File Description
src/git.ts Adds two new clone options and adjusts repo location + credential helper config behavior.
package.json Version bump to 0.2.0.
package-lock.json Lockfile version bump alignment to 0.2.0.

Copilot's findings

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

  • Files reviewed: 2/3 changed files
  • Comments generated: 2

Comment thread src/git.ts
Comment thread src/git.ts
@Chuxel Chuxel merged commit f2bb953 into main Jun 10, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants